Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance orchestration template parameter type support #105

Merged
merged 3 commits into from
Oct 4, 2017

Conversation

bzwei
Copy link
Contributor

@bzwei bzwei commented Sep 29, 2017

Better support non-string parameter types including comma-delimited-list, number, boolean

https://bugzilla.redhat.com/show_bug.cgi?id=1489908

depends on ManageIQ/manageiq#16047

Update:
The work now also include better support Cloudformation template. So it is able to accept both HOT and Cloudformation templates in either JSON or YAML format.

The last commit fixed an error in spec test that references to a factory that has been removed from manageiq repo.

It was accidently left blank when moved from manageiq repo
@bzwei bzwei force-pushed the ot_parameters branch 3 times, most recently from bcd73e2 to bce829b Compare September 29, 2017 19:37
@bzwei
Copy link
Contributor Author

bzwei commented Sep 29, 2017

@miq-bot add_label enhancement
@miq-bot bug
@miq-bot assign @aufi
cc @Ladas

@miq-bot
Copy link
Member

miq-bot commented Sep 29, 2017

@bzwei unrecognized command 'bug', ignoring...

Accepted commands are: add_label, assign, close_issue, move_issue, remove_label, rm_label, set_milestone

@bzwei
Copy link
Contributor Author

bzwei commented Sep 29, 2017

@miq-bot add_label bug

@bzwei
Copy link
Contributor Author

bzwei commented Oct 3, 2017

@miq-bot add_label wip

@miq-bot miq-bot changed the title Enhance orchestration template parameter type support [WIP] Enhance orchestration template parameter type support Oct 3, 2017
@miq-bot miq-bot added the wip label Oct 3, 2017
Better support non-string parameter types including comma-delimited-list,
number, boolean

https://bugzilla.redhat.com/show_bug.cgi?id=1489908
@bzwei
Copy link
Contributor Author

bzwei commented Oct 3, 2017

@miq-bot remove_label wip

@miq-bot
Copy link
Member

miq-bot commented Oct 3, 2017

Checked commits bzwei/manageiq-providers-openstack@0c1cdc4~...618d9d5 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 0 offenses detected
Everything looks fine. 🍪

@miq-bot miq-bot changed the title [WIP] Enhance orchestration template parameter type support Enhance orchestration template parameter type support Oct 3, 2017
@miq-bot miq-bot removed the wip label Oct 3, 2017
Copy link
Contributor

@Ladas Ladas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, nice specs :-)

@tzumainn tzumainn merged commit 79f50bb into ManageIQ:master Oct 4, 2017
@aufi aufi added this to the Sprint 71 Ending Oct 16, 2017 milestone Oct 4, 2017
@bzwei bzwei deleted the ot_parameters branch October 4, 2017 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants